Skip to content
This repository was archived by the owner on Jan 30, 2025. It is now read-only.

Conversation

inancgumus
Copy link
Collaborator

@inancgumus inancgumus commented Nov 5, 2024

What?

  • Moves BrowserContext waitForEvent Sobek-option parsing into the mapping layer.
  • Removes common.WaitForEventOptions as it's only used in the mapping.

Why?

See "Desobekifying Sobek transformation" in grafana/k6#4219.

Checklist

  • I have performed a self-review of my code
  • I have added tests for my changes
  • I have commented on my code, particularly in hard-to-understand areas

Related PR(s)/Issue(s)

#1270

@inancgumus inancgumus added refactor stability runtime stability improvements labels Nov 5, 2024
@inancgumus inancgumus self-assigned this Nov 5, 2024
@inancgumus inancgumus changed the base branch from main to desobekify/browser-context-grant-permission-options-parsing November 5, 2024 20:11
@inancgumus inancgumus force-pushed the desobekify/browser-context-waitForEvent-options branch from 738de64 to efb1ad6 Compare November 5, 2024 20:13
@inancgumus inancgumus force-pushed the desobekify/browser-context-grant-permission-options-parsing branch from 6bcbb03 to df900b8 Compare November 5, 2024 20:19
@inancgumus inancgumus force-pushed the desobekify/browser-context-waitForEvent-options branch from efb1ad6 to 1840be2 Compare November 5, 2024 20:22
@inancgumus inancgumus force-pushed the desobekify/browser-context-grant-permission-options-parsing branch from df900b8 to 998bb01 Compare November 5, 2024 20:23
@inancgumus inancgumus force-pushed the desobekify/browser-context-waitForEvent-options branch 2 times, most recently from 733ab54 to b112c6f Compare November 5, 2024 20:24
@inancgumus inancgumus force-pushed the desobekify/browser-context-grant-permission-options-parsing branch from 998bb01 to 74e5505 Compare November 5, 2024 20:59
@inancgumus inancgumus force-pushed the desobekify/browser-context-waitForEvent-options branch 2 times, most recently from 853819b to a1619b9 Compare November 5, 2024 21:10
ankur22
ankur22 previously approved these changes Nov 6, 2024
Copy link
Collaborator

@ankur22 ankur22 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM 🚀

@inancgumus inancgumus marked this pull request as ready for review November 6, 2024 15:48
Base automatically changed from desobekify/browser-context-grant-permission-options-parsing to main November 6, 2024 15:51
@inancgumus inancgumus dismissed ankur22’s stale review November 6, 2024 15:51

The base branch was changed.

@inancgumus inancgumus force-pushed the desobekify/browser-context-waitForEvent-options branch from a1619b9 to e582707 Compare November 6, 2024 15:52
@inancgumus inancgumus merged commit 4634741 into main Nov 6, 2024
19 of 23 checks passed
@inancgumus inancgumus deleted the desobekify/browser-context-waitForEvent-options branch November 6, 2024 15:59
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

refactor stability runtime stability improvements

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants